thanks for this PR - your first commit looks good. then you merged in master, where someone else had already reduced the line length to 154 (in commit #3130) in a slightly different way - so now there isn't much left of your PR! but thats ok.

so you can make the line length even smaller, to end up with a merged PR for issue #3105 - or you can move on to looking at the issues tagged outreachy: https://github.com/Parsl/parsl/issues?q=is%3Aissue+is%3Aopen+label%3Aoutreachy to look for stuff you'd like to learn more about/work on.

Thank you for the feedback, @benclifford

Sure, I'll make the further line change in the issue #3105 and then proceed to other issues tagged as outreachy.

Hey @benclifford

I made a new commit to reduce the maximum length further.
Please guide me and review the change.

ok, mostly looks good - there are a few unnecessary empty lines added that you can see here in https://github.com/Parsl/parsl/pull/3131/files in slurm.py and in db_manager.py. Those probably come from your first bit of work on this branch where you made the first changes a little bit differently than the PR that got merged (PR #3130). It would be good to remove those changes because they make this PR look more messy than it needs to be, and that can be awkward when someone looks back through git history.

Appreciate the guidance, @benclifford
You're an amazing mentor. I've resolved the changes as per your feedback, and I hope it looks good now.

Thank you.